Skip to content

GH-49444: [C++][FlightRPC][ODBC] Disable DSN default values on MacOS#49402

Merged
lidavidm merged 1 commit intoapache:mainfrom
Bit-Quill:gh-49387-macos-dsn-default
Mar 9, 2026
Merged

GH-49444: [C++][FlightRPC][ODBC] Disable DSN default values on MacOS#49402
lidavidm merged 1 commit intoapache:mainfrom
Bit-Quill:gh-49387-macos-dsn-default

Conversation

@justing-bq
Copy link
Contributor

@justing-bq justing-bq commented Feb 26, 2026

Rationale for this change

iodbc driver manager on macOS treats default values as real values when reading from system DSN, and the driver manager ignores the actual values.

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting review Awaiting review labels Feb 26, 2026
@lidavidm
Copy link
Member

@justing-bq this seems to be linked to a closed issue, can we link it to a fresh/open issue?

@alinaliBQ
Copy link
Collaborator

Hi @lidavidm I have raised #49444 for this PR, I don't have access to change the PR. Could you please help changing the PR to link to issue #49444?

@lidavidm lidavidm changed the title GH-49393: [C++][FlightRPC][ODBC] Disable DSN default values on MacOS GH-49444: [C++][FlightRPC][ODBC] Disable DSN default values on MacOS Mar 4, 2026
@lidavidm lidavidm linked an issue Mar 4, 2026 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Mar 4, 2026

⚠️ GitHub issue #49444 has been automatically assigned in GitHub to PR creator.

@alinaliBQ
Copy link
Collaborator

@lidavidm Thanks! Let me know if you have other comments for this PR

@alinaliBQ alinaliBQ force-pushed the gh-49387-macos-dsn-default branch from f9848ef to dcde0d4 Compare March 5, 2026 23:53
@lidavidm
Copy link
Member

lidavidm commented Mar 6, 2026

Is there an issue open for the tests timing out on Windows?

@alinaliBQ
Copy link
Collaborator

alinaliBQ commented Mar 6, 2026

@lidavidm I raised an issue here: #49465. I think arrow-flight-test (Timeout) failure is caught on MSVC Windows platform, and I think it is not likely caused by ODBC-related changes as we haven't modified the Flight code.

@alinaliBQ alinaliBQ force-pushed the gh-49387-macos-dsn-default branch from dcde0d4 to d064e87 Compare March 9, 2026 18:43
Co-authored-by: alinalibq <alina.li@improving.com>
@alinaliBQ alinaliBQ force-pushed the gh-49387-macos-dsn-default branch from d064e87 to e169614 Compare March 9, 2026 20:23
@lidavidm lidavidm merged commit 8c2d93c into apache:main Mar 9, 2026
56 of 61 checks passed
@lidavidm lidavidm removed the awaiting merge Awaiting merge label Mar 9, 2026
@github-actions github-actions bot added the awaiting merge Awaiting merge label Mar 9, 2026
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 3 benchmarking runs that have been run so far on merge-commit 8c2d93c.

There was 1 benchmark result indicating a performance regression:

The full Conbench report has more details. It also includes information about 6 possible false positives for unstable benchmarks that are known to sometimes produce them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[C++][FlightRPC][ODBC] Disable DSN default values on MacOS

3 participants